Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tigrc: use color15 for cursor by default (bright white) #870

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

blueyed
Copy link
Contributor

@blueyed blueyed commented Sep 26, 2018

"bold" does not necessarily mean "bright", and default white is hardly
visible on green (using solarized colors myself).

Ref: kovidgoyal/kitty#197

Could also be changed in https://github.com/blueyed/tig/blob/4e03fe46a9cc48500abc9e70a54d4d734ab335ce/contrib/chocolate.theme.tigrc#L8 maybe. /cc @edi9999

"bold" does not necessarily mean "bright", and default white is hardly
visible on green (using solarized colors myself).

Ref: kovidgoyal/kitty#197
@edi9999
Copy link
Contributor

edi9999 commented Sep 27, 2018

Yes, agree with you to change chocolate.

Can you include it in your PR ?

@blueyed
Copy link
Contributor Author

blueyed commented Sep 27, 2018

Ok.
Will wait for Jonas to approve it first.

@ThomasHaddad
Copy link

color15 is not necessarily the brightest white you have, it only depends on your colorscheme right?

Mine for example:

capture d ecran 2018-10-08 a 17 10 09

@blueyed
Copy link
Contributor Author

blueyed commented Oct 8, 2018

@ThomasHaddad
Is that a base16 based scheme?
Anyway, color15 is at least as bright as color07 for you (the same?), and we cannot assume people have more that 16 colors (configured in a certain way).
(I have a base16 based scheme, and color21 is dark black for me (in the light variant))

@ThomasHaddad
Copy link

ThomasHaddad commented Oct 8, 2018 via email

@@ -365,7 +365,7 @@ color " Acked-by" yellow default
color " Tested-by" yellow default
color " Reviewed-by" yellow default
color default default default normal
color cursor white green bold
color cursor color15 green bold
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd prefer to use the ANSI color names in the default configuration. These days I am using white on red, inspired by Mutt if I remember correctly, would that be acceptable?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ANSI color names

That is only 0-7 then?

white on red

Does not look good for me:

2019-01-16-004140_958x120_escrotum

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants